index.js ➔ ??? - Code Metrics - Inspection of "Import" - yoanm/async-response-aggregator-server - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Pull Request — master (#2)

by Yo

created 2017-02-26 11:04 UTC

index.js ➔ ??? A

↳ Parent: index.js

Complexity

Conditions	3
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes	5
Bugs	0	Features	0

Metric	Value
cc	3
c	5
b	0
f	0
nc	2
nop	2
dl	0
loc	10
rs	9.4285

"use strict";

if (['dev', 'prod'].indexOf(process.env.NODE_ENV)) {
    console.warn(`Undefined node environment "${process.env.NODE_ENV}", fallback to dev !`);
    process.env.NODE_ENV = 'dev';
}

const ON_DEATH = require('death')({
    SIGTERM: false,
    SIGQUIT: false
});

const logger = require('./lib/logger');
const taskLoggerFactory = require('./lib/logger/taskLogger');
const server = require('./lib/server');
const pkg = require('./package.json');
const NestedError = require('nested-error-stacks');

const appName = pkg.name;
const taskLogger = taskLoggerFactory(appName);

ON_DEATH((signal, err) => {

    let exitCode = 0;
    if (signal === 'SIGTERM' || signal === 'SIGQUIT') {
        exitCode = 1;
    }

    console.log('ON_DEATH');


    cleanAndExit(exitCode);
});

process.on('unhandledRejection', (reason, p) => {
    console.log('Unhandled Rejection at: Promise', p, 'reason:', reason);


    return cleanAndExit(1);
});


const cleanAndExit = (exitCode = 0) => {
    taskLogger.stopping();
    const _exit = error => {
        if (error instanceof Error) {
            console.error(error);
        }
        taskLogger.stopped();
        process.exit(exitCode);

    };

    return server.stop()
        .then(_exit, _exit);
};

taskLogger.starting();

return server.start()
    .then(() => taskLogger.started())
    .catch(error => {
        const newError = new NestedError(`Exit ${appName} after an error at initialisation`, error);

        logger.error(error.stack);

        return Promise.reject(cleanAndExit(1));
    });


1			"use strict";
2
3			if (['dev', 'prod'].indexOf(process.env.NODE_ENV)) {
4			console.warn(`Undefined node environment "${process.env.NODE_ENV}", fallback to dev !`);
5			process.env.NODE_ENV = 'dev';
6			}
7
8			const ON_DEATH = require('death')({
9			SIGTERM: false,
10			SIGQUIT: false
11			});
12
13			const logger = require('./lib/logger');
14			const taskLoggerFactory = require('./lib/logger/taskLogger');
15			const server = require('./lib/server');
16			const pkg = require('./package.json');
17			const NestedError = require('nested-error-stacks');
18
19			const appName = pkg.name;
20			const taskLogger = taskLoggerFactory(appName);
21
22			ON_DEATH((signal, err) => {
			0 ignored issues – show Unused Code introduced 2017-02-26 11:06 UTC by Report Bug Copy Issue Report The parameter `err` is not used and could be removed. This check looks for parameters in functions that are not used in the function body and are not followed by other parameters which are used inside the function. Loading history...
23			let exitCode = 0;
24			if (signal === 'SIGTERM' \|\| signal === 'SIGQUIT') {
25			exitCode = 1;
26			}
27
28			console.log('ON_DEATH');
			0 ignored issues – show Debugging Code introduced 2017-02-26 11:06 UTC by Report Bug Copy Issue Report `console.log` looks like debug code. Are you sure you do not want to remove it? Loading history...
29
30			cleanAndExit(exitCode);
31			});
32
33			process.on('unhandledRejection', (reason, p) => {
34			console.log('Unhandled Rejection at: Promise', p, 'reason:', reason);
			0 ignored issues – show Debugging Code introduced 2017-02-26 11:06 UTC by Report Bug Copy Issue Report `console.log` looks like debug code. Are you sure you do not want to remove it? Loading history...
35
36			return cleanAndExit(1);
37			});
38
39
40			const cleanAndExit = (exitCode = 0) => {
41			taskLogger.stopping();
42			const _exit = error => {
43			if (error instanceof Error) {
44			console.error(error);
45			}
46			taskLogger.stopped();
47			process.exit(exitCode);
			1 ignored issue – show Compatibility Debugging Code Best Practice introduced 2017-02-26 11:06 UTC by Report Bug Copy Issue Report Use of `process.exit()` is discouraged as it will potentially stop the complete node.js application. Consider quitting gracefully instead by throwing an Error. Loading history...
48			};
49
50			return server.stop()
51			.then(_exit, _exit);
52			};
53
54			taskLogger.starting();
55
56			return server.start()
57			.then(() => taskLogger.started())
58			.catch(error => {
59			const newError = new NestedError(`Exit ${appName} after an error at initialisation`, error);
			0 ignored issues – show Unused Code introduced 2017-02-26 11:06 UTC by Report Bug Copy Issue Report The constant `newError` seems to be never used. Consider removing it. Loading history...
60			logger.error(error.stack);
61
62			return Promise.reject(cleanAndExit(1));
63			});
64

yoanm / async-response-aggregator-server

Pull Request — master (#2)

index.js ➔ ??? A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like